Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[#59] 이슈 라벨 추가 #91

Merged
merged 14 commits into from
May 5, 2024
Merged

[#59] 이슈 라벨 추가 #91

merged 14 commits into from
May 5, 2024

Conversation

minsang-alt
Copy link
Contributor

📄 Summary

  • AssigneeDto에 관련해서 리펙터링이 필요한 로직이 많아 수정하고 카카오 이미지 Url 추가하는등의 작업을 했습니다.
  • 이슈 조회, 스프린트 조회 등 여러 곳에서 라벨이 필요하기때문에 다양한 곳에서 라벨을 추가하는 작업을 추가했습니다.
  • 스프린트들 조회시 스프린트 상태를 반환을 안해서 간단하게 추가했습니다
  • 그외에 Factory 부분에서 약간의 리펙터링을 진행했는데.. 아직도 리펙터링할 부분이 많은 것 같습니다.

🕰️ Actual Time of Completion

6시간

🙋🏻 More

minsang-alt added 12 commits May 4, 2024 15:27
@minsang-alt
feat: 이슈 라벨 Enum 추가 및 컨버터 등록
aced01b
@minsang-alt
refactor: assigneeDto from 메서드 사용 및 카카오 이미지 url 추가
5ba8996
@minsang-alt
refactor: assigneeDto member 패키지 밑으로 이동하면서 리펙터링
261e2b3
@minsang-alt
feat(EpicFactory): 라벨 추가
97b1783
@minsang-alt
feat: 라벨 추가 및 생성자 관련 SuperBuilder로 리펙터링
75d788f
@minsang-alt
Merge branch 'main' of https://github.com/AgileHub-DQ/Backend into fe…
d30507d 
…at/59
@minsang-alt
refactor: superBuilder 지우고 생성자로 다시 변경
74e0f20 
story와 task 같은 경우 생성할때 추가적인 로직이 존재하기 때문에 단순화를 위해 생성자로 다시 사용합니다
@minsang-alt
feat: 라벨 추가 및 assigneeDto관련 리펙터링
fbe01fe
@minsang-alt
style: assigneeDto 패키지 임포트 경로 수정
54c62f0
@minsang-alt
refactor(SimpleIssueResponse): label 추가
888d912
@minsang-alt
feat: storyFactory, taskFactory label 추가
a3f6630
@minsang-alt
refactor: 스프린트들 조회시 스프린트 상태도 반환하도록 수정
7a6ed28
@minsang-alt minsang-alt added 🌱 기능추가 새로운 기능 요청입니다 🦅 사소한 수정 오타 수정, prettier 적용 등 사소한 수정 🔨 리팩터링 개선사항입니다 labels May 5, 2024
@minsang-alt minsang-alt requested a review from Enble May 5, 2024 08:20
@minsang-alt minsang-alt self-assigned this May 5, 2024
@minsang-alt minsang-alt linked an issue May 5, 2024 that may be closed by this pull request
이슈에 라벨을 추가한다 #59
Closed
2 tasks
@github-actions GitHub Actions
Copy link

github-actions bot commented May 5, 2024 •
edited
Loading

Unit Test Results

44 tests   44 ✔️  2s ⏱️
20 suites    0 💤
20 files      0 ❌

Results for commit cfa93f8.

♻️ This comment has been updated with latest results.

@minsang-alt
Copy link
Contributor Author

minsang-alt commented May 5, 2024 •
edited
Loading

  • 리뷰 컨펌되면 이슈테이블에 label 칼럼 추가

Enble
Enble approved these changes May 5, 2024
View reviewed changes
Copy link
Contributor

@Enble Enble left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
local의 ddl-auto는 create로 유지하는건가요?

@minsang-alt
Copy link
Contributor Author

ddl은 update로 유지하는데 라벨 칼럼때문에 추가하라고 일부러 create로 놓았습니다

@minsang-alt minsang-alt merged commit af14c00 into main May 5, 2024
3 checks passed
@minsang-alt minsang-alt deleted the feat/59 branch May 5, 2024 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Enble Enble Enble approved these changes

Assignees

@minsang-alt minsang-alt

Labels
🦅 사소한 수정 오타 수정, prettier 적용 등 사소한 수정 🔨 리팩터링 개선사항입니다 🌱 기능추가 새로운 기능 요청입니다
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

이슈에 라벨을 추가한다
2 participants
@minsang-alt @Enble